-
Notifications
You must be signed in to change notification settings - Fork 229
Block resuming a partition merge schema change #4936
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Coding style check: Error. ⚠.
Smoke testing: Error ⚠.
Cbuild submission: Success ✓.
Regression testing: 6/564 tests failed ⚠.
The first 10 failing tests are:
fdb_push [setup failure]
mismatch_class_remsql_fdbpushredirect_generated [setup failure]
phys_rep_perf
sc_timepart
lostwrite
truncatesc_offline_generated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
.
…e sharing the destination newdb, which involves more complicated code changes and will be adressed in 8.1. Signed-off-by: Dorin Hogea <dhogea@bloomberg.net>
17c4d7c
to
05d31ff
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Coding style check: Error. ⚠.
Smoke testing: Success ✓.
Cbuild submission: Error ⚠.
Regression testing: 3/568 tests failed ⚠.
The first 10 failing tests are:
phys_rep_perf
incremental_backup_load
truncatesc_offline_generated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Coding style check: Error. ⚠.
Smoke testing: Success ✓.
Cbuild submission: Success ✓.
Regression testing: 5/568 tests failed ⚠.
The first 10 failing tests are:
analyze_fastinit_race
incoherent_slow
phys_rep_perf
incremental_backup_load
truncatesc_offline_generated
@morgando thank you |
Resuming would require sharing the destination newdb, which involves more complicated code changes and will be adressed in 8.1.